Skip to content

Added display results function to utils.py#15

Merged
jayasanka-sack merged 1 commit intomainfrom
npnorman-util-display-results
Mar 5, 2026
Merged

Added display results function to utils.py#15
jayasanka-sack merged 1 commit intomainfrom
npnorman-util-display-results

Conversation

@npnorman
Copy link
Collaborator

@npnorman npnorman commented Mar 3, 2026

No description provided.

@npnorman npnorman requested a review from jayasanka-sack March 3, 2026 21:17
@github-actions
Copy link

github-actions bot commented Mar 3, 2026

🔒 Security Test Results

Tests completed. View the security dashboard for detailed CVSS scores and trends.

📊 Download artifacts

Copy link
Member

@jayasanka-sack jayasanka-sack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, @npnorman! Could you add some detail to the description about the specific problem this solves and the approach you took? It’ll be helpful for future reference and documentation.

@npnorman
Copy link
Collaborator Author

npnorman commented Mar 4, 2026

This PR consolidates the functions of:

print(f"CVSS Base Score: {cvss_score}")
print(f"Severity Rating: {severity}")

into a function called display_results().

These print statements need to be exactly formatted like this for the workflow to pick them up using regex. This is why I standardized how displaying results should be by making a new function.

Copy link
Member

@jayasanka-sack jayasanka-sack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @npnorman!

@jayasanka-sack jayasanka-sack merged commit 468facb into main Mar 5, 2026
1 check passed
@jayasanka-sack jayasanka-sack deleted the npnorman-util-display-results branch March 5, 2026 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants